-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#5982 [ja]Addressed the translation omissions #5983
base: main
Are you sure you want to change the base?
#5982 [ja]Addressed the translation omissions #5983
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your contribution, @shukawam! I added minor requests but other than those, LGTM.
@@ -0,0 +1,110 @@ | |||
{{/* cSpell:ignore cond */ -}} [OpenTelemetry Collector](/docs/collector/)にテレメトリーを送信し、正しくエクスポートされることを確認してください。本番環境でコレクターを使用することはベストプラクティスです。テレメトリーを可視化するために、[Jaeger](https://jaegertracing.io/)、[Zipkin](https://zipkin.io/)、 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
一旦「OpenTelemetryコレクター」で統一してるので修正してください
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OpenTelemetryコレクターに修正しました
|
||
{{ end -}} | ||
|
||
エクスポーターの中でも、[OpenTelemetry Protocol (OTLP)][OTLP]エクスポーターは、OpenTelemetryのデータモデルを考慮して設計されており、OTelデータを情報の損失なく出力します。さらに、多くのテレメトリデータを扱うツールがOTLPに対応しており(例えば、[Prometheus]、[Jaeger]やほとんどの[ベンダー])、必要なときに高い柔軟性を提供します。OTLPについて詳細に学習したい場合は、[OTLP仕様][OTLP]を参照してください。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
例えば→たとえば 開く形で統一しています
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
「たとえば」に修正しました
#5982
I have addressed the translation omissions in the exporter section.
There are still some English parts within the shortcode, but I plan to handle those during the translation of other sections that use the shortcode.
Executed
npm run fix:all
in my local env and confirmed that there are no issues.